You don't need to have email as a constant in your select for your cursor, just use the value that is already there.

The bigger question I would ask is do you really need a separate email for each sku or would a single email with all the skus be acceptable? If a single email with multiple skus would be ok you don't even need the cursor. And do you mean for these emails to be only a subject and not have a body?

--EDIT--

Meant to post the cleaned up version of your cursor.

DECLARE @emailTo VARCHAR(255),

@email VARCHAR(max),

@sku VARCHAR(30),

@emailSubject VARCHAR(128),

SET @emailTo = 'myemail@mycompany.co.uk'

SET @emailSubject = 'Warning stock is below safety for : '

-- SELECT RECORDS THAT NEED TO BE EMAILED

DECLARE c1 CURSOR FOR

SELECT rtrim(StockCode) as StockCode

FROMdbo.tbl_STOCK_safety_check

WHEREemail_status = 1

-- LOOP THROUGH RECORDSET AND SEND EMAIL FOR EACH RECORD

OPEN c1

FETCH NEXT FROM c1 INTO @sku

WHILE @@FETCH_STATUS <> -1

BEGIN

EXEC msdb.dbo.sp_send_dbmail

@importance = 'High',

@recipients= @email ,

@subject = @emailSubject + @sku

FETCH NEXT FROM c1 INTO @email,@sku

END

CLOSE c1

DEALLOCATE c1